UI: Improve error handling for template upload notifications #12412
Conversation
Damans227
changed the title
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.20 #12412 +/- ##
=========================================
Coverage 16.24% 16.24%
- Complexity 13378 13381 +3
=========================================
Files 5657 5657
Lines 498971 498973 +2
Branches 60561 60561
=========================================
+ Hits 81033 81035 +2
- Misses 408904 408905 +1
+ Partials 9034 9033 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR improves error handling for template upload failures by properly extracting and displaying error messages from the SSVM (Secondary Storage VM). Instead of showing a generic "Request failed with status code 400" message, the UI now displays the actual error message from the backend.
Changes:
- Replaced custom error notification in template upload with
$notifyError()utility - Refactored error message extraction logic to prioritize response headers and data over generic error messages
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ui/src/views/image/RegisterOrUploadTemplate.vue | Simplified error handling to use $notifyError() utility |
| ui/src/utils/plugins.js | Improved error message extraction logic to check response data before falling back to generic error message |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| desc = error.response.headers['x-description'] | ||
| } | ||
| if (desc === '' && error.response.data) { | ||
| } else if (error.response.data) { |
There was a problem hiding this comment.
The condition check else if (error.response.data) is too broad and will evaluate to true for empty objects or arrays. This could lead to unnecessary processing of the response data block when there's no actual data. Consider using a more specific check like else if (error.response.data && Object.keys(error.response.data).length > 0) or checking for truthy values that actually contain data.
| } else if (error.response.data) { | |
| } else if (error.response.data && (typeof error.response.data !== 'object' || Object.keys(error.response.data).length > 0)) { |
| } else if (typeof error.response.data === 'string') { | ||
| desc = error.response.data |
There was a problem hiding this comment.
String responses should be trimmed before assignment to avoid displaying empty or whitespace-only error messages. Consider using desc = error.response.data.trim() and only assigning if the trimmed result is non-empty.
3 participants
Description
This PR fixes template upload error messages not being displayed in the UI. Previously, when template uploads failed (e.g., file format mismatch), the UI would show a generic "Request failed with status code 400" message instead of the actual error message from the SSVM (Secondary Storage VM).
Fixes: #11513
Types of changes
Bug Severity
Screenshots (if appropriate):
Broken:
Fixed:
How Has This Been Tested?